-
-
Notifications
You must be signed in to change notification settings - Fork 699
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[13.0][MIG] repair #2721
[13.0][MIG] repair #2721
Conversation
87f4e8a
to
2fe141e
Compare
2fe141e
to
2f28c43
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs post-migration file with:
openupgrade.load_data(
env.cr, "repair", "migrations/13.0.1.0/noupdate_changes.xml")
2f28c43
to
3119fd7
Compare
3119fd7
to
8e2d2f4
Compare
addons/repair/migrations/13.0.1.0/openupgrade_analysis_work.txt
Outdated
Show resolved
Hide resolved
8e2d2f4
to
f4053a4
Compare
Thanks @pedrobaeza , changes done. |
f4053a4
to
b96992d
Compare
53c623d
to
ededd14
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you mistook
f380d6e
to
aff700d
Compare
@victoralmau I do not understand you mistook or do not understand that this commit:f380d6e is correct and this one is useless (why need to update legacy name) aff700d |
My last changes are necessary to prevent error in github action called "Test OpenUpgrade migration / test (pull_request)" |
aff700d
to
b784004
Compare
b784004
to
02b2f26
Compare
f36d32a
to
856bb7f
Compare
Thanks @kos94ok-3D and @MiquelRForgeFlow changes done. |
Poor @victoralmau 😂 |
Well, it's a mix of bad interpretations + incorrect reviews. The history is:
So please all take more attention for not having all these rounds. |
cb24500
to
dd38f24
Compare
dd38f24
to
e4873e1
Compare
Thanks @pedrobaeza changes done. |
e4873e1
to
5d57149
Compare
5d57149
to
ec98fca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please double check changes
Migration to 13.0.
@Tecnativa TT27870